Skip to content

[Partner Nodes] always display the custom width and height in GPTImage2#13651

Merged
Kosinkadink merged 1 commit intomasterfrom
chore/api-nodes/GPTImage-rmv-adv-w
May 1, 2026
Merged

[Partner Nodes] always display the custom width and height in GPTImage2#13651
Kosinkadink merged 1 commit intomasterfrom
chore/api-nodes/GPTImage-rmv-adv-w

Conversation

@bigcat88
Copy link
Copy Markdown
Contributor

@bigcat88 bigcat88 commented May 1, 2026 •
edited
Loading

API Node PR Checklist

Scope

  • Is API Node Change

Pricing & Billing

  • Need pricing update
  • No pricing update

If Need pricing update:

  • Metronome rate cards updated
  • Auto‑billing tests updated and passing

QA

  • QA done
  • QA not required

Comms

  • Informed Kosinkadink

@bigcat88
chore(api-nodes): always display the custom width and height in GPTIm…
2e7b7b6 
…age2 node

Signed-off-by: bigcat88 <bigcat88@icloud.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 1, 2026 •
edited
Loading

📝 Walkthrough

Walkthrough

The OpenAIGPTImage1 node's schema is updated by removing the advanced=True flag from the custom_width and custom_height input field definitions. This makes these fields appear as standard UI inputs rather than advanced inputs in the node interface. No validation logic, request/response behavior, or other input parameters are modified by this change.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title references 'GPTImage2' but the actual change is to 'OpenAIGPTImage1' in the file, making the title misleading about which component was modified. Update the title to accurately reflect that the change is to OpenAIGPTImage1, not GPTImage2, for example: '[Partner Nodes] always display the custom width and height in OpenAIGPTImage1'
✅ Passed checks (4 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description check ✅ Passed The PR description is related to the changeset, confirming this is an API Node change that requires no pricing or QA updates, matching the technical scope of removing the 'advanced' flag from width/height inputs.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Kosinkadink Kosinkadink merged commit 96f1cee into master May 1, 2026
17 checks passed
@bigcat88 bigcat88 deleted the chore/api-nodes/GPTImage-rmv-adv-w branch May 1, 2026 06:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Kosinkadink Kosinkadink Kosinkadink approved these changes

@comfyanonymous comfyanonymous Awaiting requested review from comfyanonymous comfyanonymous is a code owner

@guill guill Awaiting requested review from guill guill is a code owner

@alexisrolland alexisrolland Awaiting requested review from alexisrolland alexisrolland is a code owner

@rattus128 rattus128 Awaiting requested review from rattus128 rattus128 is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bigcat88 @Kosinkadink